Skip to content
This repository was archived by the owner on Jan 18, 2024. It is now read-only.

Add permissions to Role in timescaledb-single, fixing #599#601

Open
geilername wants to merge 2 commits intotimescale:mainfrom
geilername:main
Open

Add permissions to Role in timescaledb-single, fixing #599#601
geilername wants to merge 2 commits intotimescale:mainfrom
geilername:main

Conversation

@geilername
Copy link
Copy Markdown

@geilername geilername commented Mar 29, 2023

What this PR does / why we need it

Add permission to create and modify services to the Role used in timescaledb-single chart.

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

Special notes for your reviewer

Checklist

Signed-off-by: Maximilian Marschall <maximilian.marschall@3xm.at>
Signed-off-by: Maximilian Marschall <maximilian.marschall@3xm.at>
@geilername geilername requested a review from a team as a code owner March 29, 2023 13:56
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 29, 2023

CLA assistant check
All committers have signed the CLA.

@joschiwald
Copy link
Copy Markdown
Contributor

That issue is already fixed on patroni side. You could use the patroni-dcs-failsafe tags of the timescale/timescaledb-ha images, which uses a more current patroni version.

@hongbo-miao
Copy link
Copy Markdown

hongbo-miao commented Apr 28, 2023

Thanks the information from @joschiwald ! 😃

I succeed using new patroni-dcs-failsafe tag by

helm upgrade \
  timescale \
  timescaledb-single \
  --install \
  --repo=https://charts.timescale.com \
  --namespace=my-timescale \
  --create-namespace \
  --values=my-values.yaml

my-values.yaml

image:
  repository: timescale/timescaledb-ha
  tag: pg15.2-ts2.10.2-patroni-dcs-failsafe-p0

without any errors.

So maybe just need fix by updating tag instead.

Copy link
Copy Markdown

@RLutsch RLutsch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

had the same issue, this fixed it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TimescaleDB Single ServiceAccount missing create:service permission

5 participants